-
Notifications
You must be signed in to change notification settings - Fork 1.1k
PYTHON-5421 continued - update changelog, update docstring, and add testing #2420
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It may just be me but I would do options
instead of 'options' what do you think? Also I'd say "parse_uri
's options
parameter" instead of parse_uri's
options
. </nit>
Totally makes sense, i'm kinda terrible with words sometimes so i'll happily suggest any revisions to my text :) |
pymongo/asynchronous/uri_parser.py
Outdated
@@ -80,6 +80,9 @@ async def parse_uri( | |||
wait for a response from the DNS server. | |||
:param srv_service_name: A custom SRV service name | |||
.. versionchanged:: 4.14 | |||
'options' is now type ``dict`` as opposed to a ``_CaseInsensitiveDictionary``. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here too!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry missed this, good catch!
just options
right?
pymongo/synchronous/uri_parser.py
Outdated
@@ -80,6 +80,9 @@ def parse_uri( | |||
wait for a response from the DNS server. | |||
:param srv_service_name: A custom SRV service name | |||
.. versionchanged:: 4.14 | |||
'options' is now type ``dict`` as opposed to a ``_CaseInsensitiveDictionary``. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And here
follow up PR to #2413